-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Misc: Bump Yad AppImage to use new release #1077
Conversation
151d9e7
to
844287f
Compare
We can probably remove the duplicate |
The AppImage appears to work fine on my Linux PC. I don't have SteamTinkerLaunch on my Steam Deck, but I should be able to download and test the Yad AppImage standalone there sometime next week. Ideally before this gets merged, someone will test it out on SteamOS. |
Are you sure the AppImage downloaded correctly? It doesn't look like it did based on your terminal output. Double-check that the filesize is ~80mb. I was able to run it on my Steam Deck and it errored out around the glibc version. I'm guessing there's some glibc incompatibility between my Arch PC and the Steam Deck. SteamOS Sigh, I'll probably have to get a Debian VM or something to build against, assuming it has an older glibc version. Alternatively, this could wait until SteamOS updates to a newer glibc version. Hopefully with the next snapshot update this happens. |
You were right the file downloaded with curl was 0 bytes... Maybe we can compile it in a VM for Steamos or my idea of a CI will simplify this situation. |
Once again, I don't think CI is necessary. This also isn't an urgent change, Yad is not updated frequently. If needs be we can manage building it on an Ubuntu VM once every two years :-) |
b49f8ba
to
5287973
Compare
Bump Yad AppImage to use new release over at: https://github.com/sonic2kk/steamtinkerlaunch-tweaks/releases/tag/Yad-fbfb0fa-x86_64.AppImage
5287973
to
faf56c7
Compare
Updated AppImage, needs testing. @AtomHare feel free to check it out on SteamOS if you like :-) |
I just tried it and it worked well ! Both in desktop and in gamemode ! However I uncountered an issue, I did a |
Network errors can occur and cause this. I'm not sure we can do much about this without some logic to retry downloads, but failure to download/connect to the internet is quite common on SteamOS (iirc it has a lot to do with the Steam Client downloading in the background hogging the connection and causing timeouts). Thanks for testing though! This can be merged then I think. |
The download didn't fail because of a "steam deck unreliability", it's completely unrelated indeed, it looks like the crafted URL by STL is : but now it's at I'm going to create a little PR for this ! |
Good to know that this issue is separate, thanks for catching this! Although it's still important to note (for the few Steam Deck users that actually do things properly and check previous issues and PRs) that dependency download failures are incredibly common on SteamOS for various projects, including SteamTinkerLaunch. However this case is separate indeed, but, the download may still fail. New release has been published here following sonic2kk/steamtinkerlaunch-tweaks#4: https://github.com/sonic2kk/steamtinkerlaunch-tweaks/releases/tag/Yad-13.0-x86_64.AppImage |
No issue at all, glad to help
Honestly, I never got download issues with my Steam Deck, maybe it's more related to people having bad Wi-Fi connections at home |
Hi, I updated STL on my Steam Deck, updated ~/.config/steamtinkerlaunch/uris.conf and removed yad executable to force redownload of new version. Couldn't thing of another procedure. Then i ran steamtinkerlaunch in terminal and it proceeded to download yad but because this: |
Huh, that's interesting. Thanks for pointing that out. I'll push a fix for that now. I guess it was needed after all, but I'm not sure why. I guess maybe the values in |
Resolves issue pointed out in https://github.com/sonic2kk/steamtinkerlaunch/pull/1077\#issuecomment-2144032796
Should be fixed with 2d0b0ec. |
Implements #1060.
Related #859.
Bump Yad AppImage to use new release over at: https://github.com/sonic2kk/steamtinkerlaunch-tweaks/releases/tag/Yad-fbfb0fa-x86_64.AppImage
We pin to the latest AppImage release on GitHub by assuming the release name and AppImage will have the same string. The new release should follow this convention. This means existing code and older releases will still use the old AppImage, but newer versions will download the new AppImage.
This new release updates the Yad version. As always on SteamOS, dependencies will have to be cleared to test this (the
steamtinkerlaunch cleardeckdeps
command).This PR needs significant amounts of testing before it can be merged. Once merged, the AppImage release will no longer be marked as a draft.